Skip to content

Replace founder bulk delete with safer single-business flow#78

Merged
DevCalebR merged 1 commit into
mainfrom
codex/safer-single-business-delete
May 26, 2026
Merged

Replace founder bulk delete with safer single-business flow#78
DevCalebR merged 1 commit into
mainfrom
codex/safer-single-business-delete

Conversation

@DevCalebR

Copy link
Copy Markdown
Owner

UI changes\n- replaces the main founder reset action with a new Delete one test/demo business tool\n- adds a selectable founder-only business list with business name, owner email, real-vs-test/demo status, archive state, and permanent-delete warning\n- requires typing the exact business name before the Delete this business button enables\n- moves Delete all current businesses into an advanced founder reset disclosure instead of the primary destructive action\n\n## Safety rules\n- keeps hard delete limited to archived test/demo businesses through the existing server-side eligibility checks\n- keeps real customer businesses on the archive-first path and shows explicit Archive real customers copy\n- preserves founder/admin authorization, tenant isolation, and existing cascade deletion behavior\n- keeps redirects outside catchable redirect-error paths so normal feedback does not surface NEXT_REDIRECT\n\n## Tests added\n- verifies the single-business delete card is exact-name gated and keeps hard delete narrow\n- verifies deleting one archived demo/test business leaves another business intact\n- verifies the delete action still requires admin auth, exact-name confirmation, and has no redirect-catching pattern\n- verifies the founder reset UI now prioritizes single-business delete and keeps bulk delete in advanced\n\n## Validation\n- npm run typecheck\n- npm run lint\n- npm test\n- npm run build\n\n## Remaining risks\n- the exact-name enable/disable behavior is client-side, so the server action still remains the final authority for confirmation and eligibility\n- the advanced bulk delete action still exists for founder use; it is de-emphasized and still protected, but it remains intentionally available for last-resort cleanup

@vercel

vercel Bot commented May 26, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
callbackcloser Ready Ready Preview, Comment May 26, 2026 6:11am

Request Review

@DevCalebR DevCalebR merged commit 7123b07 into main May 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant